🦋 Changeset detected

Latest commit: 6cc2a1c

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Deploying graph-tooling with    Cloudflare Pages

View logs

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Is this PR finished?
graph build with these changes doesn't work if id type is set to Int8.
Codegen needs to be adjusted.

@YaroShkvorets PR is finished now

@shiyasmohd
Codegen now works, but scaffolded subgraph doesn't.

Steps to reproduce:

1. Init new subgraph:

graph init test test --from-contract 0xdAC17F958D2ee523a2206206994597C13D831ec7 --protocol ethereum --network mainnet --index-events

2. Replace id: Bytes! for Issue entity in the GraphQL schema with id: Int8!
3. Run graph codegen - works
4. Run graph build - fails with

 Compile subgraphERROR TS2322: Type '~lib/@graphprotocol/graph-ts/common/collections/Bytes' is not assignable to type 'i64'.

     event.transaction.hash.concatI32(event.logIndex.toI32())
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 in src/contract.ts(30,5)

Basically need to generate different code for Int8 that constructs unique 64-bit id for our event:

let entity = new Issue(
    event.transaction.hash.concatI32(event.logIndex.toI32())
  )

I guess we could just hash this thing and take first 8 bytes in these cases.

Or we could construct something from block number, transaction index and event index.